Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rick/improve sourcemap url instructions (#2459) #2460

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

rick-bt
Copy link
Collaborator

@rick-bt rick-bt commented Sep 26, 2023

  • Improve instruction for creating an upload URL
  • Update source map instructions to use config file
  • Remove global flag for plugin installation
  • Provide alt path for integrating source code.

BT-668 Update Source Maps doc

Description

  1. The upload URL is critical to source maps. The current link to generating the URL is somewhat hidden, and not enough context is provided.
  2. Support for a configuration file has been extended, and is now the preferred integration method.

Motivation and Context

Users were failing to correctly generate an upload URL.
The configuration file will improve integration success.

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation fix (typos, incorrect content, missing content, etc.)

* Improve instruction for creating an upload URL

* Update source map instructions to use config file

* Remove global flag for plugin installation

* Provide alt path for integrating source code.
@github-actions
Copy link

Deploy preview ready for 2460!
https://docs.dev.saucelabs.net
built from 81b7a2f

@rick-bt rick-bt merged commit 590c948 into main Sep 26, 2023
@rick-bt rick-bt deleted the rick/BT-668-Update-Source-Maps-doc branch September 26, 2023 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants